Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat!: upgrade bulk_beta_modify_access to drf ( 30 ) #35604

Merged
merged 13 commits into from
Dec 17, 2024

Conversation

awais786
Copy link
Contributor

@awais786 awais786 commented Oct 7, 2024

#35285

Existing code

verify using postman

  1. Post request using this URL

  2. pass form-data

    identifiers: audit,awaisq
    email_students: true
    auto_enroll: true
    action: add or remove

expected result

{
    "action": "add",
    "results": [
        {
            "identifier": "audit",
            "error": false,
            "userDoesNotExist": false,
            "is_active": true
        },
        {
            "identifier": "awaisq",
            "error": false,
            "userDoesNotExist": false,
            "is_active": true
        }
    ]
}

end to end testing

  1. go to this page instructor > membership
  2. pass valid usernames with comma separated e.g admin, staff
  3. Press add beta testers button.
  4. Now from below dropdown Select a course team role:: select beta testers.
  5. It will load the admin and staff as beta testers.
  6. Now repeat above steps 2 and press remove beta tester button.
  7. Now from below dropdown Select a course team role:: select beta testers.
    8.It will appear empty.
Screenshot 2024-08-09 at 7 33 28 PM

@awais786 awais786 requested a review from feanil October 7, 2024 16:36
@awais786 awais786 changed the title Bulk beta modify access to drf feat!: upgrade bulk_beta_modify_access to drf ( 30 ) Oct 7, 2024
@awais786 awais786 marked this pull request as ready for review October 9, 2024 08:10
@awais786 awais786 force-pushed the bulk_beta_modify_access-to-drf branch from 4a84b4d to 6719f15 Compare December 16, 2024 10:08
@awais786 awais786 merged commit 53de406 into master Dec 17, 2024
50 checks passed
@awais786 awais786 deleted the bulk_beta_modify_access-to-drf branch December 17, 2024 11:35
@edx-pipeline-bot
Copy link
Contributor

2U Release Notice: This PR has been deployed to the edX staging environment in preparation for a release to production.

@edx-pipeline-bot
Copy link
Contributor

2U Release Notice: This PR has been deployed to the edX production environment.

1 similar comment
@edx-pipeline-bot
Copy link
Contributor

2U Release Notice: This PR has been deployed to the edX production environment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants